Skip to content

Conversation

@eivindj-nordic
Copy link
Contributor

@eivindj-nordic eivindj-nordic commented Sep 19, 2025

No description provided.

@eivindj-nordic eivindj-nordic self-assigned this Sep 19, 2025
@eivindj-nordic eivindj-nordic requested review from a team as code owners September 19, 2025 06:08
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Sep 19, 2025
@github-actions
Copy link

You can find the documentation preview for this PR here.

@eivindj-nordic eivindj-nordic requested a review from a team as a code owner September 19, 2025 06:27
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 19, 2025
@github-actions github-actions bot removed the doc-required PR must not be merged without tech writer approval. label Sep 26, 2025
@eivindj-nordic eivindj-nordic requested a review from a team as a code owner October 21, 2025 11:59
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 21, 2025
@eivindj-nordic eivindj-nordic force-pushed the hid_test branch 2 times, most recently from 57251d0 to d39b06b Compare October 21, 2025 12:22
evt_rw_auth = &ble_evt.evt.gatts_evt.params.authorize_request;

switch (auth_evt_type) {
case ON_OTHER_TYPE: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the parenthesis? We usually tend to omit them

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will remove.

TEST_ASSERT_EQUAL(NRF_ERROR_NULL, err);
}


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra newline

/* Exit Suspend command. */
#define HIDS_CONTROL_POINT_EXIT_SUSPEND 0x01


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra newline.

@eivindj-nordic eivindj-nordic force-pushed the hid_test branch 2 times, most recently from 6d83510 to 1f501e8 Compare November 12, 2025 14:03
Add unit tests for ble_hids service.

Signed-off-by: Eivind Jølsgard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants